Skip to content

Conversation

@christian-oreilly
Copy link
Member

@christian-oreilly christian-oreilly commented Feb 6, 2025

fix #2

@christian-oreilly christian-oreilly linked an issue Feb 6, 2025 that may be closed by this pull request
6 tasks
@christian-oreilly
Copy link
Member Author

@Sabah98 Although the goal of the PR is to add continuous integration, I also addressed some packaging issues. For example, the most common structure is to have the source code nested in a folder of the name of the package (not "src"). Also, all modules required a __init__.py, even if this file is left empty.

@christian-oreilly
Copy link
Member Author

The tests on this PR won't pass but I will by-pass them when ready to merge because I don't want to spell check this before you merge your dev branch.

@christian-oreilly
Copy link
Member Author

@dw42CSCE This branch is now rebased against the main so you can git pull --rebase an add you changes to this branch directly and push.

@Sabah98 Please mind that this branch rename the src folder into a contourusv folder. I think I explained these changes to you in previous message. So, when/if you make changes and push them, you may encounter some conflicts. Let me know if you need assistance solving these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add continuous integration

4 participants